Skip to content

Conversation

@sander1095
Copy link
Contributor

@sander1095 sander1095 commented Feb 18, 2025

I have also removed the --preview flag as it seems unnecessary. It's not necessary to add a preview package to get OpenAPI to be AOT compatible in .NET 9.

Fixes #34756


Internal previews

📄 File 🔗 Preview link
aspnetcore/fundamentals/openapi/aspnetcore-openapi.md Generate OpenAPI documents

…nning .NET 9 or lower

I also removed the --prerelease flag from the package, no reason to add a prerelease version; AOT works fine without prerelease
@sander1095
Copy link
Contributor Author

Currently I've added aspnetcore-10.0 as the moniker range. I assume that my PR for improving the webapiaot template will go live in preview 2.

I've looked through several commits to find out how I could set it to this preview-2 value, but I can't figure this out.

If there's any changes required, let me know!

@sander1095
Copy link
Contributor Author

Important note:

This should probably only get merged when dotnet/aspnetcore#60337 is merged.

@Rick-Anderson Rick-Anderson marked this pull request as draft February 19, 2025 23:31
Copy link
Contributor

@Rick-Anderson Rick-Anderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:::moniker range="= aspnetcore-9.0" is preferred over <10

@Rick-Anderson Rick-Anderson changed the title Do not show unnecessary steps for creating AOT-compatible OpenAPI application when running in .NET 10 DO NOT MERGE: Do not show unnecessary steps for creating AOT-compatible OpenAPI application when running in .NET 10 Feb 25, 2025
@sander1095 sander1095 marked this pull request as ready for review March 11, 2025 15:20
@sander1095
Copy link
Contributor Author

Hi @Rick-Anderson ! The main PR has been merged: dotnet/aspnetcore#60337

I've changed this PR's title and draft status, I believe it can be merged :)?

@sander1095 sander1095 changed the title DO NOT MERGE: Do not show unnecessary steps for creating AOT-compatible OpenAPI application when running in .NET 10 Do not show unnecessary steps for creating AOT-compatible OpenAPI application when running in .NET 10 Mar 11, 2025
@Rick-Anderson Rick-Anderson merged commit 3050b58 into dotnet:main Mar 11, 2025
3 checks passed
@Rick-Anderson
Copy link
Contributor

@sander1095 thanks so much for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change OpenAPI AOT/Trimming as .NET 10 will require less steps to achieve this

2 participants